O2 linter results: ❌ 14 errors, ⚠️ 10 warnings, 🔕 2 disabled

Error while checking build/O2Physics/o2 for 2af6f3a at 2025-11-21 23:46:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/13931-slc9_x86-64/0/PWGJE/Tasks/jetOutlierQA.cxx:692:15: error: unused variable 'mcCollisionIDcoll' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/13931-slc9_x86-64/0/PWGJE/Tasks/jetOutlierQA.cxx:693:15: error: unused variable 'mcCollisionIDOutlier' [-Werror=unused-variable]
ninja: build stopped: subcommand failed.

Full log here.

@jaimenorman Please always make sure your branch compiles without warnings and that it's properly formatted before opening the PR for review.

@jaimenorman Please always make sure your branch compiles without warnings and that it's properly formatted before opening the PR for review.

sorry, indeed it compiled locally so didn't pick up on the mistake. I've opened a new PR ( #14001 ), so close this one

@jaimenorman Please always make sure your branch compiles without warnings and that it's properly formatted before opening the PR for review.

sorry, indeed it compiled locally so didn't pick up on the mistake. I've opened a new PR ( #14001 ), so close this one

Why did you not just update this PR instead?

@jaimenorman Please always make sure your branch compiles without warnings and that it's properly formatted before opening the PR for review.

sorry, indeed it compiled locally so didn't pick up on the mistake. I've opened a new PR ( #14001 ), so close this one

Why did you not just update this PR instead?

I had made some additional changes to the task in the meantime - I thought it would be cleaner to push both together so I did this in a new PR

@jaimenorman Please always make sure your branch compiles without warnings and that it's properly formatted before opening the PR for review.

sorry, indeed it compiled locally so didn't pick up on the mistake. I've opened a new PR ( #14001 ), so close this one

Why did you not just update this PR instead?

I had made some additional changes to the task in the meantime - I thought it would be cleaner to push both together so I did this in a new PR

I actually doesn't have any benefits to replace a PR with another one. It will just run all tests from scratch which might be unnecessary. You can modify all properties of the PR (title, description, commits).